Skip to content

Improve spelling#8901

Merged
bramkragten merged 4 commits intohome-assistant:devfrom
LEJOUI:patch-1
Apr 13, 2021
Merged

Improve spelling#8901
bramkragten merged 4 commits intohome-assistant:devfrom
LEJOUI:patch-1

Conversation

@LEJOUI
Copy link
Copy Markdown
Contributor

@LEJOUI LEJOUI commented Apr 12, 2021

Breaking change

Proposed change

The ... of this line keeps bugging me. It looks unfinished. I'd like to change it in a line that is at least complete.
Here's my proposal. Feedback is appreciated.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

Comment thread src/translations/en.json Outdated
@LEJOUI
Copy link
Copy Markdown
Contributor Author

LEJOUI commented Apr 12, 2021

Thanks for your feedback!
I'm also thinking that one of these two may fit

  1. Manage integrations with services or devices
  2. Manage integrations that link with services or devices

@spacegaier
Copy link
Copy Markdown
Member

spacegaier commented Apr 12, 2021

"Manage integrations that" always sounds as if only a specific subset of all integrations are in there, when if fact all are in there.

=> I would vote for alternative 1: Manage integrations with services or devices (could also be "and")

@frenck
Copy link
Copy Markdown
Member

frenck commented Apr 12, 2021

It is or an integration can only expose a single type; as in either device or service.

@bramkragten bramkragten merged commit 96bc3ef into home-assistant:dev Apr 13, 2021
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants